Skip to content

[호돌이] Week2 #729

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Dec 22, 2024
Merged

[호돌이] Week2 #729

merged 6 commits into from
Dec 22, 2024

Conversation

yeeZinu
Copy link
Contributor

@yeeZinu yeeZinu commented Dec 17, 2024

답안 제출 문제

체크 리스트

  • 우측 메뉴에서 PR을 Projects에 추가해주세요.
  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 바로 앞에 PR을 열어주신 분을 코드 검토자로 지정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

@yeeZinu yeeZinu added the js label Dec 17, 2024
@yeeZinu yeeZinu self-assigned this Dec 17, 2024
@yeeZinu yeeZinu requested a review from a team as a code owner December 17, 2024 01:05
@yeeZinu yeeZinu requested a review from Chaedie December 17, 2024 01:06
*/
var climbStairs = function (n) {
// n+1 배열을 만들고 0으로 초기화
const dp = new Array(n + 1).fill(0);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

배열 선언에서 .fill(0) 부분은 없어도 괜찮지 않을까 궁금합니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정수가 들어갈 자리라서 미리 0으로 초기화했었는데 안해도 상관은 없을거같긴 합니다.
한번 찾아보겠습니다!

};

// Map을 사용해서 데이터 최신화
let count = new Map();
Copy link
Contributor

@TonyKim9401 TonyKim9401 Dec 21, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

map 대신 알파벳 만큼인 26짜리 배열 하나만을 사용해서 풀이하는 방법도 있을것 같아요!

Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2주차 문제풀이 고생 많으셨습니다!
문제에 따라서 최적화 과정을 설명과 함께 적어주셔서 이해하는데 더 도움이 된 것 같아요.
3주차도 파이팅입니다!

@Chaedie
Copy link
Contributor

Chaedie commented Dec 22, 2024

리뷰로 도움드리진 못했습니다만, 3주차도 같이 화이팅 하면 좋을것 같습니다..!
고생 많으셨습니다..!

@SamTheKorean SamTheKorean merged commit da886dd into DaleStudy:main Dec 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

4 participants